Conversation
|
|
||
| public String titular; | ||
| public Double cantidad; | ||
|
|
There was a problem hiding this comment.
Estas dos propiedades no deberían ser publicas si no primadas para encapsular el código dentro de la clase
|
|
||
| public Cuenta(String getTitular, Double getCant) { | ||
| this.titular = getTitular; | ||
| this.cantidad = getCant; |
There was a problem hiding this comment.
debería controlarse que la cantidad nunca sea menor a 0
|
|
||
| public void retirar(Double cant) { | ||
|
|
||
| if(cant > this.cantidad){ |
There was a problem hiding this comment.
Esto esta mal planteado , puesto que plantea que solo se podrá retirar dinero si la cantidad a retirar es mayor a la cantidad existente, debería de plantearse de la siguiente manera:
Si la cantidad a retirar menos la cantidad existente no es menor que 0.
|
|
||
| public Cuenta cuenta; | ||
|
|
||
| public void main(String[] args) { |
There was a problem hiding this comment.
es " public static void main(String[] args) {"
| public void main(String[] args) { | ||
| // TODO code application logic here | ||
|
|
||
| cuenta = new Cuenta("Jorge Lopez"); |
There was a problem hiding this comment.
Para probar los dos constructores lo mejor seria crear dos objetos Cuenta uno con titular y otro con titular y cantidad
| } | ||
|
|
||
| private char comprobarSexo(char sexo) { | ||
| if (sexo != 'M' || sexo != 'H') { |
There was a problem hiding this comment.
Debería de ser && no || , puesto que H ya es diferente de M , y entraría si solo se pusiera H.
2 participants
New pull with Cuenta class